View GitHub Repository node-patternlibrary

Schedule Inspection
Subscribe
Passed
Push — master ( 55414e...5f39f4 )
by Björn
03:25
created

T_FUNCTION ➔ getPatterns   B

↳ Parent: Project

Complexity

Conditions 6
Paths 4

Size

Total Lines 1

Duplication

Lines 1
Ratio 100 %

Code Coverage

Tests 0
CRAP Score 42

Importance

Changes 0
Metric Value
dl 1
loc 1
ccs 0
cts 0
cp 0
rs 8.8571
c 0
b 0
f 0
cc 6
nc 4
crap 42
nop 1
1 
/**
2 
 * retrieve current pattern-library data
3 
 */
4 View Code Duplication 
function getPatterns ( type_or_category ) {
0 ignored issues
– show
Duplication introduced by
This code seems to be duplicated in your project.
Loading history...
5 225 
    if (type_or_category) {
6 125 
        var patterns = {};
7 125 
        for (var patternkey in this.data.patterns) {
0 ignored issues
– show
Complexity introduced by
A for in loop automatically includes the property of any prototype object, consider checking the key using hasOwnProperty.

When iterating over the keys of an object, this includes not only the keys of the object, but also keys contained in the prototype of that object. It is generally a best practice to check for these keys specifically:

var someObject;
for (var key in someObject) {
    if ( ! someObject.hasOwnProperty(key)) {
        continue; // Skip keys from the prototype.
    }

    doSomethingWith(key);
}
Loading history...
8 
            var pattern = this.data.patterns[patternkey];
9 2 
            if (
10 
                ( String(patternkey).indexOf(type_or_category) != -1 ) ||
11 
                ( pattern.pattern.categories && ( pattern.pattern.categories.indexOf(type_or_category) != -1 ) )
12 
            ) {
13 
                patterns[patternkey] = pattern;
14 
            }
15 
        }
16 125 
        return patterns;
17 
    }
18
19 100 
    return this.data.patterns;
20 

            


            
                                                                                                            
                            
            
                                    
            
            

                
                    21
                

                                    
                                                    
                
                
                
}

            


            
                                                                                                            
                                                                
            
                                    
            
            

                
                    22
                

                                    
                                                    
                
                
                

            


            
                                                        
            
                                    
            
            

                
                    23
                

                                    
                                                    
                
                
                
module.exports = getPatterns;